fix(settings): shortcut recorder stops on an outside click or window blur (#1272) - #1308

Fixes #1272.

Found by the Stage 3 C4 hunt. Plan: docs/plans/2026-09-25-shortcut-recorder-release.md (first commit).

Why

While recording a command shortcut, a capture-phase window listener swallows every key. It released only on Escape or a finished chord. So clicking the Settings search box and typing t silently saved T as that command's shortcut.

What merges

• Recording ends on a mousedown outside the recorder's own button, or when the window blurs. The outside click matches HotkeyInput and MouseButtonInput. Ending on blur is stricter than HotkeyInput, which only clears held modifiers, because a recording left armed across an app switch is how a stray key got saved.
• The listener runs in the capture phase, so a control that stops propagation cannot keep the recording armed.
• The recorder button toggles itself: a click on it releases the recording and re-arms it rather than ending it.
• Only the list's scrollbar is exempt (review B). The list has 134 rows in a 420 px scroller, and grabbing its scrollbar is not leaving. Only a press past the scroller's client box counts as the scrollbar (onScrollbar). A click in the scroller's own 8 px gaps between category blocks, or on a row, still ends the recording (review B, round 2).

Tests

CommandKeybindingsRow.capture.renderer.test.tsx renders the real component with the real store. It covers:

• an outside click, which ends the recording without swallowing or saving t (red on main);
• a blur, which ends it (red on main);
• a scrollbar drag, which keeps it;
• a gap click and a row click, which end it;
• the recorder toggle;
• a stopPropagation control.

Every mutation reviewers listed dies. npx tsc -b is clean.

🤖 Generated with Claude Code